Skip to content

[tests] remove test_pipelines_common.py - #14669

Open
sayakpaul wants to merge 15 commits into
mainfrom
remove-test-pipelines-common
Open

sayakpaul wants to merge 15 commits into
mainfrom
remove-test-pipelines-common

Conversation

@sayakpaul

@sayakpaul sayakpaul commented Sep 1, 2026

Copy link
Copy Markdown
Member
  • Move FluxIPAdapterTesterMixin to the IP Adapter testing utils since it's shared by more than Flux.
  • Migrate old imports in the test suites of DeepFloyd IF, etc. to use new ones.
  • Migrate the Stable Diffusion Adapter test suite to use new mixins.
  • Remove test_pipelines_common.py.

@github-actions github-actions Bot added tests size/L PR with diff > 200 LOC labels Sep 1, 2026
# support `"pt"`; adding that here is a `src/` change and out of scope for this test migration, so the affected
# tests are marked `xfail` rather than skipped: whoever adds the `"pt"` branch will see them XPASS and can drop
# these markers.
NO_PT_OUTPUT = pytest.mark.xfail(

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Open to other ideas but I chose to xfail these tests and not introduce anything really new to our shared tester mixins.

@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/stable_diffusion_adapter/test_stable_diffusion_adapter.py

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

pytest tests/pipelines/stable_diffusion_adapter/test_stable_diffusion_adapter.py passed on GPU — view logs.

Comment thread tests/pipelines/testing_utils/memory.py Outdated
Comment thread tests/pipelines/kandinsky/test_kandinsky_prior.py Outdated
@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/stable_diffusion_2/test_stable_diffusion_depth.py::TestStableDiffusionDepth2ImgPipelineMemory::test_group_offloading_inference

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

⚠️ pytest tests/pipelines/stable_diffusion_2/test_stable_diffusion_depth.py::TestStableDiffusionDepth2ImgPipelineMemory::test_group_offloading_inference was cancelled on GPU — view logs.

@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/consisid/test_consisid.py tests/pipelines/stable_audio/test_stable_audio.py

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

pytest tests/pipelines/consisid/test_consisid.py tests/pipelines/stable_audio/test_stable_audio.py passed on GPU — view logs.

@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/stable_diffusion_2/test_stable_diffusion_depth.py::TestStableDiffusionDepth2ImgPipelineMemory::test_group_offloading_inference

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

pytest tests/pipelines/stable_diffusion_2/test_stable_diffusion_depth.py::TestStableDiffusionDepth2ImgPipelineMemory::test_group_offloading_inference passed on GPU — view logs.

@github-actions github-actions Bot added the CI label Sep 1, 2026
pytest -n 1 --max-worker-restart=0 --dist=loadfile \
--make-reports=tests_torch_minimum_version_cuda \
tests/models/test_modeling_common.py \
tests/pipelines/test_pipelines_common.py \

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No longer exists.

Comment on lines -17 to +19
- "tests/pipelines/test_pipelines_common.py"
- "tests/pipelines/testing_utils/**.py"
- "tests/models/test_modeling_common.py"
- "tests/models/testing_utils/**.py"

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To have parity in our CI when test changes happen.

pytest -n 1 --max-worker-restart=0 --dist=loadfile \
--make-reports=tests_torch_minimum_cuda \
tests/models/test_modeling_common.py \
tests/pipelines/test_pipelines_common.py \

@sayakpaul sayakpaul Sep 3, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No longer exists.

)


class AdapterPipelineTesterConfig(BasePipelineTesterConfig):

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The remaining test suite being refactored.

@sayakpaul

Copy link
Copy Markdown
Member Author

/diffusers-bot pytest tests/pipelines/kandinsky/ tests/pipelines/kandinsky2_2/

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

pytest tests/pipelines/kandinsky/ tests/pipelines/kandinsky2_2/ passed on GPU — view logs.

Comment thread tests/pipelines/testing_utils/ip_adapter.py Outdated
Comment thread tests/pipelines/flux/testing_utils.py
@sayakpaul
sayakpaul requested a review from DN6 September 11, 2026 05:28
@hf-security-analysis

hf-security-analysis Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

⚠️ Workflow security review — 20 high, 15 medium

Scanned 3 workflow file(s) with zizmor, pinact, OSV/GHSA, Claude, in full.

.github/workflows/nightly_tests.yml

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:34 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:71 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:117 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:189 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:233 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:282 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:355 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:411 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/nightly_tests.yml:469 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:97 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:98 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:99 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:103 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/nightly_tests.yml:104 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

.github/workflows/pr_tests_gpu.yml

  • ⚠️ HIGH unpinned-images — .github/workflows/pr_tests_gpu.yml:91 (via zizmor)
    unpinned image references — container image uses the floating 'latest' tag

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/pr_tests_gpu.yml:130 (via zizmor)
    unpinned image references — container image uses the floating 'latest' tag

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/pr_tests_gpu.yml:194 (via zizmor)
    unpinned image references — container image uses the floating 'latest' tag

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/pr_tests_gpu.yml:264 (via zizmor)
    unpinned image references — container image uses the floating 'latest' tag

    Fix: unpinned image references

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:164 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:166 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:167 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:172 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:179 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/pr_tests_gpu.yml:180 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

.github/workflows/release_tests_fast.yml

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:34 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:74 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:119 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:176 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:234 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:277 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • ⚠️ HIGH unpinned-images — .github/workflows/release_tests_fast.yml:320 (via zizmor)
    unpinned image references — container image is unpinned

    Fix: unpinned image references

  • 🟡 MEDIUM template-injection — .github/workflows/release_tests_fast.yml:100 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/release_tests_fast.yml:101 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/release_tests_fast.yml:105 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

  • 🟡 MEDIUM template-injection — .github/workflows/release_tests_fast.yml:106 (via zizmor)
    code injection via template expansion — may expand into attacker-controllable code

    Fix: Map the expression into the step's env: block and reference the shell variable instead of interpolating it into run:.

ℹ️ No change could be applied automatically.

Pinning and static findings are deterministic. Findings marked (via Claude) are model judgements — check them before acting.

fix(ci): harden workflow files flagged on #14669

Co-authored-by: hf-security-analysis[bot] <265538906+hf-security-analysis[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI size/L PR with diff > 200 LOC tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants